Skip to content

Add agent skills for AI coding#2804

Merged
yihao03 merged 17 commits into
MarkBind:masterfrom
yihao03:feat/experiment-with-skills
Mar 25, 2026
Merged

Add agent skills for AI coding#2804
yihao03 merged 17 commits into
MarkBind:masterfrom
yihao03:feat/experiment-with-skills

Conversation

@yihao03

@yihao03 yihao03 commented Jan 26, 2026

Copy link
Copy Markdown
Contributor

used opencode to read devGuide and generated relevant skills

What is the purpose of this pull request?

  • Documentation update
  • Bug fix
  • Feature addition or enhancement
  • Code maintenance
  • DevOps
  • Improve developer experience
  • Others, please explain:

Overview of changes:
This PR serves as part of the effort to develop an AI workflow for development.

We opted to use the open source SKILL.md standard as it is provider agnostic and has good community support.
Therefore we could adopt mainstream and popular workflows developed by others without being restricted to a specific provider (e.g. ClaudeCode).

Useful references in creating skills:

Anything you'd like to highlight/discuss:

Testing instructions:
Use AI Coding tools like OpenCode/ClaudeCode to start writing code. Check if the skills are being picked up
correctly and performs as desired

Proposed commit message: (wrap lines at 72 characters)
Add Agent Skills.


Checklist: ☑️

  • Updated the documentation for feature additions and enhancements
  • Added tests for bug fixes or features
  • Linked all related issues
  • No unrelated changes

Reviewer checklist:

Indicate the SEMVER impact of the PR:

  • Major (when you make incompatible API changes)
  • Minor (when you add functionality in a backward compatible manner)
  • Patch (when you make backward compatible bug fixes)

At the end of the review, please label the PR with the appropriate label: r.Major, r.Minor, r.Patch.

Breaking change release note preparation (if applicable):

  • To be included in the release note for any feature that is made obsolete/breaking

Give a brief explanation note about:

  • what was the old feature that was made obsolete
  • any replacement feature (if any), and
  • how the author should modify his website to migrate from the old feature to the replacement feature (if possible).

used opencode to read devGuide and generated relevant skills
@codecov

codecov Bot commented Jan 26, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.29%. Comparing base (510231a) to head (93036b3).
⚠️ Report is 3 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2804   +/-   ##
=======================================
  Coverage   70.29%   70.29%           
=======================================
  Files         129      129           
  Lines        6951     6951           
  Branches     1573     1510   -63     
=======================================
  Hits         4886     4886           
  Misses       2059     2059           
  Partials        6        6           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@yihao03 yihao03 mentioned this pull request Jan 26, 2026
14 tasks
@yihao03 yihao03 marked this pull request as ready for review February 1, 2026 14:50
@gerteck

gerteck commented Feb 2, 2026

Copy link
Copy Markdown
Member

The references would definitely be useful for an AI agent to immediately have a good grasp of the codebase.

However, I am worried about the maintainability of the references.

  • Would having these references (e.g. referencing specific Node versions, method names, file names and directory structures) cause a maintainence overhead when we do refactoring?
  • What kind of measures do we have to make sure that the references do not drift from the features?

How do we find the balance of keeping a abstract and high level explanation of the design in the references enough to not have to add too much maintainability burden, but have it detailed enough to justify the token count and usefulness.

This is on top of the current developer documentation that has to be updated as well.

wdyt?

@yihao03

yihao03 commented Feb 3, 2026

Copy link
Copy Markdown
Contributor Author

Would it be useful if we created a subagent #2822 that updates the skills accordingly when changes are made to the devGuide? the solution to AI is... more AI...

@Harjun751 Harjun751 mentioned this pull request Feb 9, 2026
14 tasks
@yihao03

yihao03 commented Feb 9, 2026

Copy link
Copy Markdown
Contributor Author

closes #2822

@Harjun751 Harjun751 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some minor nits. Only problems are the prev/next page navigation buttons.

Also let's discuss on reverting the update-docs skill deletion

Comment thread .github/agents/update-docs.md
Comment thread .github/skills/markbind-typescript-migration/SKILL.md
Comment thread .github/skills/markbind-typescript-migration/SKILL.md Outdated
Comment thread docs/_markbind/layouts/devGuide.md
Comment thread docs/devGuide/aiUse/index.md Outdated
Comment thread docs/_markbind/layouts/devGuide.md Outdated

@MoshiMoshiMochi MoshiMoshiMochi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work on this @yihao03. Just seeking clarification and small changes on others. Majority are just flagging out the emojis so that we don't miss out on any 💯.

Comment thread docs/devGuide/aiUse/skills.md
Comment thread .github/skills/markbind-typescript-migration/SKILL.md Outdated
Comment thread .github/skills/markbind-typescript-migration/SKILL.md Outdated
Comment thread .github/skills/markbind-typescript-migration/SKILL.md Outdated
Comment thread .github/skills/markbind-typescript-migration/references/rename-commit.md Outdated
Comment thread .github/skills/markbind-typescript-migration/references/rename-commit.md Outdated
Comment thread docs/devGuide/aiUse/subagents.md Outdated
@yihao03 yihao03 requested a review from Harjun751 March 25, 2026 06:16

@Harjun751 Harjun751 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@yihao03 yihao03 merged commit 64c1dca into MarkBind:master Mar 25, 2026
10 checks passed
@github-actions

Copy link
Copy Markdown

@yihao03 Each PR must have a SEMVER impact label, please remember to label the PR properly.

@gerteck gerteck added the r.Patch Version resolver: increment by 0.0.1 label Mar 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

r.Patch Version resolver: increment by 0.0.1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants